Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with default browser prompt not hiding resolved #550

Merged
merged 1 commit into from
Apr 20, 2022

Conversation

tomasstrba
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/1177771139624306/1202041516752977/f

Description:
This PR contains a fix of default browser prompt visibility. After setting the browser as default during the onboarding, the promt on the home page remained visible

Steps to test this PR:

  1. Set Safari as your default browser in System Preferences.
  2. Navigate to about:welcome
  3. During the onboarding, make DDG browser as default
  4. When the onboarding finishes, click on + button to open the homepage
  5. Make sure the prompt to set DDG as default is not visible
  • Test with Release configuration
  • Test proper deallocation of tabs
  • Make sure committed submodule changes are desired

Internal references:

Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM

@brindy brindy self-assigned this Apr 20, 2022
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

@brindy brindy assigned tomasstrba and unassigned brindy Apr 20, 2022
@tomasstrba tomasstrba merged commit 0f3b67a into develop Apr 20, 2022
@tomasstrba tomasstrba deleted the tom/set-default-prompt branch April 20, 2022 13:19
samsymons added a commit that referenced this pull request Apr 27, 2022
# By Christopher Brind (1) and others
# Via GitHub
* develop:
  Autofill scrolling fix (#538)
  Add web-compat code (#547)
  Issue with default browser prompt not hiding resolved (#550)
  Open Autofill settings when triggering Settings from Autofill (#534)
  Revert "don't draw webview background to avoid flashes (#545)" (#549)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants